Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Follow recent tsp-typescript-client API additions for configurations #1140

Merged
merged 1 commit into from
Nov 6, 2024

Conversation

bhufmann
Copy link
Collaborator

@bhufmann bhufmann commented Nov 5, 2024

  • Adopt tsp-typescript-client v0.5.0
  • Use ProviderType enum instead of string for outputs
  • Update API for global configuration
  • Add new methods of ITspClient in TheiaRpcTspProxy

Requires eclipse-cdt-cloud/tsp-typescript-client#119 (part of tsp-typescript-client v0.5.0)

Signed-off-by: Bernd Hufmann bernd.hufmann@ericsson.com

@bhufmann bhufmann force-pushed the dp_config_updates_base branch 2 times, most recently from 335ce43 to 9ff41a7 Compare November 6, 2024 20:16
@bhufmann
Copy link
Collaborator Author

bhufmann commented Nov 6, 2024

Please note a license review of tsp-typescript-client v0.5.0 has been opened:
https://gitlab.eclipse.org/eclipsefdn/emo-team/iplab/-/issues/17102

- Adopt tsp-typescript-client v0.5.0
- Use ProviderType enum instead of string for outputs
- Update API for global configuration
- Add new methods of ITspClient in TheiaRpcTspProxy

Signed-off-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
@bhufmann bhufmann force-pushed the dp_config_updates_base branch from 9ff41a7 to 741859d Compare November 6, 2024 20:33
@bhufmann bhufmann requested a review from PatrickTasse November 6, 2024 20:33
@bhufmann bhufmann merged commit 1fd2cfe into eclipse-cdt-cloud:master Nov 6, 2024
6 of 7 checks passed
@bhufmann bhufmann deleted the dp_config_updates_base branch November 6, 2024 20:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants